Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix hassio warnings and false True (positive) for DisableLinkage feature #394

Merged
merged 2 commits into from
Oct 12, 2024

Conversation

jandusek4
Copy link
Contributor

Fix hassio warning about blocking SSL setup Detected blocking call to load_default_certs and
setup without awaiting async_forward_entry_setup, which can cause the setup lock to be released before the setup is done

Also fixed approach how to detect if DisableLinkage feature is present which return false True (positive) even when the feature is not present in the camera and direct cgi-bin call return 400 error.

setup without awaiting async_forward_entry_setup, which can cause the setup lock to be released before the setup is done. This will stop working in Home Assistant 2025.1

Detected blocking call to load_default_certs

Signed-off-by: Jan Dušek <[email protected]>
Unify the way it is done elsewhere to prevent excesive false positive retun of True when DisableLinkage is actually available and returns error from camera. After thsi fix, it is queried twice and then the integration knows it is unavailable.

Signed-off-by: Jan Dušek <[email protected]>
@rroller
Copy link
Owner

rroller commented Oct 12, 2024

Thank you!

@rroller rroller merged commit 6522072 into rroller:main Oct 12, 2024
1 of 3 checks passed
@rroller
Copy link
Owner

rroller commented Oct 12, 2024

@jandusek4
Copy link
Contributor Author

For retest, I replaced my local modified code with latest release and is is OK in logs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants